Skip to content

added admin check for dropdown#446

Merged
gpalmer27 merged 1 commit into
mainfrom
admin-button-fix
Jun 7, 2026
Merged

added admin check for dropdown#446
gpalmer27 merged 1 commit into
mainfrom
admin-button-fix

Conversation

@gpalmer27
Copy link
Copy Markdown
Collaborator

Description

see above. this was done so that @23langloisj can't see the admin page when he doesn't have admin permissions 💀

Motivation and Context

Closes #[ticket]

How has this been tested?

tested w/ admin permissions and non admin permissions and made sure on mobile i can only see the admin button on profile dropdown when i'm admin

Screenshots (if appropriate):

Screenshot 2026-05-26 at 10 22 33 PM

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Database migration
    • Ran pnpm db:generate and verified generated SQL migration files in packages/db/drizzle

Checklist:

  • My code follows the code style of this project.
  • I have moved the ticket to "In Review"
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have added tests to cover my changes.
  • All new and existing tests passed.

@vercel
Copy link
Copy Markdown

vercel Bot commented May 27, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
cooper Ready Ready Preview, Comment May 27, 2026 2:22am
2 Skipped Deployments
Project Deployment Actions Updated (UTC)
cooper-auth Skipped Skipped May 27, 2026 2:22am
cooper-docs Skipped Skipped May 27, 2026 2:22am

Request Review

@gpalmer27 gpalmer27 requested a review from songmichael11 May 27, 2026 02:23
Copy link
Copy Markdown
Collaborator

@songmichael11 songmichael11 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

good catch lgtm

@gpalmer27 gpalmer27 merged commit fcc0fa9 into main Jun 7, 2026
10 checks passed
@gpalmer27 gpalmer27 deleted the admin-button-fix branch June 7, 2026 01:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants